PED-12023: Ansible on SAP #545
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@dariavladykina please review. Thanks |
Amrita42
added a commit
that referenced
this pull request
Nov 3, 2025
* structure * structure * typo * fqdn * collection * collect-2 * collection-3 * plabook * concept-playbook * type * playbook-review * playbook * check * self review * review feedback * toc re-structure * adds aws * aws * gcp * validate and hosts * existing hosts * Fix missing tags * validation * self review * self review * tech review-2 * tech * validate * tech review-2 * tech review * tech-review * resolev conflicts * final tech-review --------- Co-authored-by: Tom Schraitle <[email protected]>
There was a problem hiding this comment.
Hi @Amrita42, please see some suggestions for language from me. Thanks!
| <!-- S T R U C T U R E --> | ||
| <structure renderas="article" xml:id="intro-sap-ansible" xml:lang="en"> | ||
| <merge> | ||
| <title>Introduction to Ansible automation for &sles4sap; 16</title> |
There was a problem hiding this comment.
Suggested change
| <title>Introduction to Ansible automation for &sles4sap; 16</title> | |
| <title>Introduction to Ansible Automation for &sles4sap; 16</title> |
| </meta> | ||
|
|
||
| <meta name="title" its:translate="yes">Introduction to Ansible automation for &sles4sap;</meta> | ||
| <meta name="description" its:translate="yes">Learn about a set of Ansible collections and playbooks designed specifically for &sles4sap; on &productname; 16.</meta> |
There was a problem hiding this comment.
Suggested change
| <meta name="description" its:translate="yes">Learn about a set of Ansible collections and playbooks designed specifically for &sles4sap; on &productname; 16.</meta> | |
| <meta name="description" its:translate="yes">Learn about a set of Ansible collections and playbooks designed specifically for &sles4sap; on &productname; 16</meta> |
|
|
||
| <meta name="title" its:translate="yes">Introduction to Ansible automation for &sles4sap;</meta> | ||
| <meta name="description" its:translate="yes">Learn about a set of Ansible collections and playbooks designed specifically for &sles4sap; on &productname; 16.</meta> | ||
| <meta name="social-descr" its:translate="yes">How to use Ansible automation for SAP environments on &productname; 16 systems. </meta> |
There was a problem hiding this comment.
Suggested change
| <meta name="social-descr" its:translate="yes">How to use Ansible automation for SAP environments on &productname; 16 systems. </meta> | |
| <meta name="social-descr" its:translate="yes">How to use Ansible automation for SAP environments on &productname; 16 systems</meta> |
| <phrase>&aarch64;</phrase> | ||
| </meta> | ||
|
|
||
| <meta name="title" its:translate="yes">Introduction to Ansible automation for &sles4sap;</meta> |
There was a problem hiding this comment.
Suggested change
| <meta name="title" its:translate="yes">Introduction to Ansible automation for &sles4sap;</meta> | |
| <meta name="title" its:translate="yes">Introduction to Ansible Automation for &sles4sap;</meta> |
| </listitem> | ||
| <listitem> | ||
| <para> | ||
| Understanding the architecture of &sap; systems. |
There was a problem hiding this comment.
Suggested change
| Understanding the architecture of &sap; systems. | |
| Understanding the architecture of &sap; systems |
| </substeps> | ||
| </step> | ||
| <step><para>Configure Passwordless SSH Authentication.</para> | ||
| <para>For automation, the Ansible control node must be able to connect to the target host as the &rootuser; without a password prompt. |
There was a problem hiding this comment.
Suggested change
| <para>For automation, the Ansible control node must be able to connect to the target host as the &rootuser; without a password prompt. | |
| <para>For automation, the Ansible control node must be able to connect to the target host as &rootuser; without a password prompt. |
| </step> | ||
| <step><para>Verify that Ansible can successfully connect to and gather facts from your target host.</para> | ||
| <screen>&prompt.sudo; ansible -i ansible_inventory_noninteractive.yml all -m ping</screen> | ||
| <para>A successful connection test shows a Ping Pong response.</para> |
There was a problem hiding this comment.
Suggested change
| <para>A successful connection test shows a Ping Pong response.</para> | |
| <para>A successful connection test shows a ping-pong response.</para> |
| <step><para>Prepare the control node.</para> | ||
| <substeps> | ||
| <step> | ||
| <para>Install the <literal>sles_sap_automation</literal> pattern that contains all recommended packages for automation.</para> |
There was a problem hiding this comment.
Suggested change
| <para>Install the <literal>sles_sap_automation</literal> pattern that contains all recommended packages for automation.</para> | |
| <para>Install the <literal>sles_sap_automation</literal> pattern, which contains all recommended packages for automation.</para> |
| <step> | ||
| <para>You can encrypt variables for playbooks in different ways to ensure security compliance.</para> | ||
| <substeps> | ||
| <step><para>Encrypt all variables for scenario. </para></step> |
There was a problem hiding this comment.
Suggested change
| <step><para>Encrypt all variables for scenario. </para></step> | |
| <step><para>Encrypt all variables in the scenario. </para></step> |
| </step> | ||
| <step><para>Use the playbook directly from <literal>/usr/share/ansible/playbooks/ansible.playbooks_for_sap/deploy_scenarios/sap_s4hana_distributed_ha</literal> without editing it. | ||
| If you need to adjust the playbook, we recommend copying it outside of the package directory. | ||
| Execute the Ansible Playbook with Ansible Vault key file.</para> |
There was a problem hiding this comment.
Suggested change
| Execute the Ansible Playbook with Ansible Vault key file.</para> | |
| Execute the Ansible playbook with the Ansible vault key file.</para> |
There was a problem hiding this comment.
Hi @Amrita42, please see some suggestions for language from me. Thanks!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR creator: Description
Describe the overall goals of this pull request.
PR creator: Are there any relevant issues/feature requests?
PR reviewer: Checklist for editorial review
Apart from the usual checks, please double-check also the following: